Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][FIX] base_import_async: restore class #563

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

danielduqma
Copy link
Contributor

In migration to 16.0, input class oe_import_queue was replaced by another one when it has to be added instead, causing the module not to work. In Javascript, input is searched by this class and it's not found, so value cannot be recovered and import is made synchronously.

This can be reproduced in runboat, trying to import any model and checking Import in the background. Records are imported in foreground.

Migration was made in #523, seems that it was merged without simple functional testing

cc @gurneyalex @dzungtran89

In migration to 16.0, input class `oe_import_queue` was replaced by
another one when it has to be added instead, causing the module not
to work. In Javascript, input is searched by this class and it's not
found, so value cannot be recovered and import is made
synchronously.
Copy link
Contributor

@asierneiradev asierneiradev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Good Work 🥇

@danielduqma danielduqma changed the title [FIX] base_import_async: restore class [16.0][FIX] base_import_async: restore class Sep 13, 2023
Copy link
Contributor

@kikopeiro kikopeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM (without code review)

Copy link
Contributor

@dzungtran89 dzungtran89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @danielduqma , thanks for the fix 🙏

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@danielduqma
Copy link
Contributor Author

@gurneyalex could you please take a look? Thanks in advance!

@kikopeiro
Copy link
Contributor

CC @dtorreskaertech

@asierneiradev
Copy link
Contributor

@guewen @ivantodorovich

Could you give us a hand?

Thank you very much

Copy link
Member

@guewen guewen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@guewen
Copy link
Member

guewen commented Sep 25, 2023

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-563-by-guewen-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d0a4dd3 into OCA:16.0 Sep 25, 2023
4 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 499fe3b. Thanks a lot for contributing to OCA. ❤️

@asierneiradev
Copy link
Contributor

Thanks!

@guewen, many thanks to you for the support and speed,

best regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants